leap0/_sync/process.py (1)

53-55: Consider extracting shared payload expansion logic.

Sync and async clients now duplicate the same interpolation branch. A shared internal helper would reduce drift risk.

♻️ Possible extraction direction

- payload: JsonObject = {"command": expand_env(command, env) if env else command}
- if cwd is not None:
-     payload["cwd"] = expand_env(cwd, env) if env else cwd
+ payload: JsonObject = build_process_payload(command=command, cwd=cwd, timeout=timeout, env=env)

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@leap0/_sync/process.py` around lines 53 - 55, The payload construction in
process.py duplicates expansion logic (using expand_env for command and cwd);
extract that into a shared helper (e.g., a function named build_command_payload
or _expand_payload) that accepts command, cwd, and env and returns the
JsonObject payload with interpolated values, then replace the inline logic in
both the sync and async clients to call this helper (refer to expand_env,
payload creation, command, cwd, env in the diff) to avoid duplication and keep
behavior identical.

README.md (1)

122-126: Show the interpolated result in the example.

The example currently assigns templated but does not show expected output, which makes the interpolation behavior less explicit for readers.

📘 Proposed doc tweak

 templated = sandbox.process.execute(
     command="echo $NAME from ${PLACE}",
     cwd="/workspace/$NAME",
     env={"NAME": "leap0", "PLACE": "sandbox"},
 )
+print(templated.stdout.strip())  # leap0 from sandbox

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@README.md` around lines 122 - 126, The example assigns templated from
sandbox.process.execute but doesn't show the interpolated output; update the
README example around the templated assignment to display the result (e.g., call
or print/return templated or add a comment showing the expected string "leap0
from sandbox") so readers see that sandbox.process.execute performed variable
interpolation on the command "echo $NAME from ${PLACE}" with
env={"NAME":"leap0","PLACE":"sandbox"} and cwd="/workspace/$NAME".